Revert "signalTracker: Use WeakMap in SignalManager to avoid leaking objects"

This reverts commit 085102be74f4c98094712a165e5128feaca50438.

We need the SignalManager map top be iterable at shutdown for some of
the following changes. A WeakMap is not iterable. This revert changes it
back to a regular Map, which re-introduces the leaks caused by this.
Those will be fixed differently by the two followup commits.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2466>

diff --git a/js/misc/signalTracker.js b/js/misc/signalTracker.js
index 8444564df4d1727de4fc17a571f2f41b85f3d22c..1c3f4f9df4d17b53ceed4d7aa56886be0a0da7ca 100644 (file)
--- a/js/misc/signalTracker.js
+++ b/js/misc/signalTracker.js
@@ -42,7 +42,7 @@ class SignalManager {
     }
 
     constructor() {
-        this._signalTrackers = new WeakMap();
+        this._signalTrackers = new Map();
     }
 
     /**